Skip to content

[v8] Command "services": Eliminate duplicate app names#3663

Merged
Samze merged 1 commit intocloudfoundry:v8from
sap-contributions:v8_services_cmd_multi_bindings
Jan 14, 2026
Merged

[v8] Command "services": Eliminate duplicate app names#3663
Samze merged 1 commit intocloudfoundry:v8from
sap-contributions:v8_services_cmd_multi_bindings

Conversation

@jochenehret
Copy link
Contributor

@jochenehret jochenehret commented Dec 9, 2025 •
edited
Loading

Description of the Change

For the "services" command: In case of multiple service bindings per (service instance guid, app guid), eliminate duplicate app names in the "bound apps" section.

Without this PR, you may see repeated app names like in this example:

# cf services
Getting service instances in org testorg / space testspace as admin...

name                                 offering        plan              bound apps                                                last operation     broker   upgrade available
service1                             fake-service    fake-async-plan   testapp3, testapp1, testapp1                              create succeeded   sb1      no
service2                             fake-service    fake-async-plan   testapp1, testapp1                                        create succeeded   sb1      no

Why Is This PR Valuable?

Eliminates redundant information in output.

Applicable Issues

https://github.com/cloudfoundry/community/blob/main/toc/rfc/rfc-0040-service-binding-rotation.md#cf-cli

How Urgent Is The Change?

Not super-urgent.

Other Relevant Parties

@jochenehret
Command "services": Eliminate duplicate app names
025733a 
* in case of multiple service bindings per (service instance guid, app guid), eliminate duplicate app names in the "bound apps" section
@stephanme stephanme mentioned this pull request Dec 9, 2025
Open
15 tasks
prkalle
prkalle approved these changes Jan 9, 2026
View reviewed changes
Copy link
Contributor

@prkalle prkalle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Samze Samze merged commit 8d188fd into cloudfoundry:v8 Jan 14, 2026
13 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Samze Samze Samze approved these changes

+2 more reviewers

@prkalle prkalle prkalle approved these changes

@Dariquest Dariquest Dariquest approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jochenehret @Samze @prkalle @Dariquest